Skip to content

fix: 修复界面交付判定与回合内纠错恢复 - #270

Merged
taekchef merged 3 commits into
omdsh-dev:mainfrom
wang-kaopu:fix/issue-263-267-fence-feedback
Oct 8, 2026
Merged

taekchef merged 3 commits into
omdsh-dev:mainfrom
wang-kaopu:fix/issue-263-267-fence-feedback

Conversation

@wang-kaopu

@wang-kaopu wang-kaopu commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #263
Closes #267

界面工具正常返回“内容不合法”时,之前会被误算为已交付;插件重新加载后,也可能丢失已发送的纠错记录,重复提醒或重新获得纠错次数。本次修复让交付判断和纠错记录在重新加载前后保持一致。

改动

  • 界面工具只有明确返回成功状态(status=rendered)才算交付,失败、工具错误和无法识别的结果继续按未交付处理。
  • 插件加载时同步读取已有会话历史,并接管新恢复的会话;保留当前回合编号,按消息 ID 恢复已用次数,避免重复记账。
  • 同一回合内相同错误只纠正一次,渲染失败与零交付提醒共同使用每轮最多两次的额度;新回合可以再次纠正相同错误。
  • 保留宿主已有的仅推理、无正式答复重试流程,更新中英文说明及变更记录。

验证

  • 本地完整检查:1029 项通过、99 项跳过,类型检查、构建及独立运行产物检查通过。
  • 独立复跑围栏反馈与插件入口测试:77 项通过;代码规范和需求复核均无实质问题。
  • 真实宿主会话测试覆盖重载恢复、初始化期间的新消息/回合结束/模型请求、跨回合相同错误、当前回合去重和两次上限。
  • 新提交的线上检查结果见本 PR 的 GitHub 检查记录。

贡献

感谢 @wang-kaopu(Kunping Wang)提供原始交付判定与重载记账修复及回归测试。维护者的补充提交追加在原提交之后,保留贡献者原始署名。

@taekchef taekchef changed the title fix: 完善 fence feedback 的交付判定与重载恢复 fix: 修复界面交付判定与回合内纠错恢复 Oct 8, 2026

@taekchef taekchef left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

谢谢 @wang-kaopu 提供交付结果判断、重载记账修复和回归测试!

已在你的原提交后补入真实会话历史接管、回合编号保留,以及仅在当前回合去重的修复。新一次提问可以再次纠正相同错误,同一回合重载后仍不会重复提醒或重新获得纠错次数。

本地完整检查 1029 项通过、99 项跳过,77 项相关测试独立复跑通过,代码规范与需求复核通过;最新提交的 7 项线上检查均通过。保留你的原始提交与署名,合并记录也会注明贡献。

@taekchef
taekchef merged commit 30a8139 into omdsh-dev:main Oct 8, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants